Skip to content

Fix any_folder missing files issue#450

Closed
a-zw wants to merge 5 commits into
eclipse-score:mainfrom
etas-contrib:fix-any-folder-missing-deps
Closed

Fix any_folder missing files issue#450
a-zw wants to merge 5 commits into
eclipse-score:mainfrom
etas-contrib:fix-any-folder-missing-deps

Conversation

@a-zw
Copy link
Copy Markdown
Contributor

@a-zw a-zw commented Mar 24, 2026 •
edited
Loading

📌 Description

The fix turns any_folder from clever little hack into a complex beast.
So, I added a concept page to explain that: https://eclipse-score.github.io/docs-as-code/pr-450/concepts/any_folder.html

🚨 Impact Analysis

  • This change does not violate any tool requirements and is covered by existing tool requirements
  • This change does not violate any design decisions
  • Otherwise I have created a ticket for new tool qualification

✅ Checklist

  • Added/updated documentation for new or changed features
  • Added/updated tests to cover the changes
  • Followed project coding standards and guidelines

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 24, 2026 •
edited
Loading

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //src:license-check

Status: ⚠️ Needs Review

Click to expand output 
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.3.0) and connecting to it...
INFO: Invocation ID: b382f906-8080-4113-87c0-1c1802af4a3c
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: src
Loading: 0 packages loaded
    currently loading: src
Analyzing: target //src:license-check (1 packages loaded)
Analyzing: target //src:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //src:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //src:license-check (68 packages loaded, 9 targets configured)

Analyzing: target //src:license-check (69 packages loaded, 9 targets configured)

Analyzing: target //src:license-check (76 packages loaded, 9 targets configured)

Analyzing: target //src:license-check (128 packages loaded, 1545 targets configured)

Analyzing: target //src:license-check (137 packages loaded, 2554 targets configured)

Analyzing: target //src:license-check (140 packages loaded, 2576 targets configured)

Analyzing: target //src:license-check (140 packages loaded, 2576 targets configured)

Analyzing: target //src:license-check (140 packages loaded, 2576 targets configured)

Analyzing: target //src:license-check (143 packages loaded, 4465 targets configured)

Analyzing: target //src:license-check (143 packages loaded, 4465 targets configured)

Analyzing: target //src:license-check (143 packages loaded, 4465 targets configured)

Analyzing: target //src:license-check (143 packages loaded, 4465 targets configured)

Analyzing: target //src:license-check (144 packages loaded, 4589 targets configured)

INFO: Analyzed target //src:license-check (145 packages loaded, 4715 targets configured).
[14 / 16] [Prepa] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar
[15 / 16] Building src/license.check.license_check.jar (); 0s disk-cache, multiplex-worker
INFO: Found 1 target...
Target //src:license.check.license_check up-to-date:
  bazel-bin/src/license.check.license_check
  bazel-bin/src/license.check.license_check.jar
INFO: Elapsed time: 25.948s, Critical Path: 2.64s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 16 total actions
INFO: Running command line: bazel-bin/src/license.check.license_check src/formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

@a-zw
Copy link
Copy Markdown
Contributor Author

a-zw commented Mar 24, 2026

I fear the whole any_folder concept is broken in combination with combo builds.

As any_folder relies on configuration in conf.py, this configuration is ignored when included in a combo build as external source.

Potential way out: any_folder works recursively and reads conf.py. Then we use it for docs_combo instead of sphinx_collections. Reading foreign conf.py files is scary though.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 25, 2026

The created documentation from the pull request is available at: docu-html

@a-zw a-zw force-pushed the fix-any-folder-missing-deps branch from fcdf69d to 5576bbe Compare March 26, 2026 13:34
@a-zw a-zw marked this pull request as ready for review March 26, 2026 14:58
@a-zw a-zw closed this Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@AlexanderLanin AlexanderLanin Awaiting requested review from AlexanderLanin AlexanderLanin is a code owner

@MaximilianSoerenPollak MaximilianSoerenPollak Awaiting requested review from MaximilianSoerenPollak MaximilianSoerenPollak is a code owner

@dcalavrezo-qorix dcalavrezo-qorix Awaiting requested review from dcalavrezo-qorix dcalavrezo-qorix is a code owner

@nradakovic nradakovic Awaiting requested review from nradakovic nradakovic is a code owner

Assignees

No one assigned

Labels

None yet

Projects

INF - Infrastructure Community
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@a-zw